Codecov Report

Merging #2079 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master    #2079   +/-   ##
=========================================
  Coverage     93.24%   93.24%           
  Complexity     3854     3854           
=========================================
  Files           215      215           
  Lines         10411    10411           
=========================================
  Hits           9708     9708           
  Misses          703      703           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5651d8a...371775b. Read the comment docs.

Ping @SamRemis. This is a blocker for Laravel.

Thank you for the PR @GrahamCampbell, and I apologize for the delay in looking into it. I want to be positive that this will have no customer impact. It definitely looks safe to me, but this does strip some of the key data from $defaultChain. I don't see anywhere the keys are consumed so it should be safe to accept. I need to double check; barring any new findings, it will be completed soon

That's why I showed you https://3v4l.org/LJchL, https://3v4l.org/IBO1U, which proves this is safe.

@GrahamCampbell I tried pushing it myself, but the repo denied permission. To add the changelog file, add the below contents to a file under .changes/nextrelease/[file-name-here].json. I used '.changes/nextrelease/php-8-patch.json'.

[
{
"type": "bugfix",
"category": "Credentials",
"description": "Fixes a crash in PHP 8.0 by calling array_values on the default chain array passed into self::chain"
}
]

You do (already) have permission to push to my branch, however the PR probably needs rebasing. Feel free to take it over, or just cherry-pick my commit.

Any status about this fix?

Just rebased this and added the nextrelease entry.